Kilka miesięcy temu dowiedziałem się, że podczas pracy z java.sql.connection muszę zamknąć PreparedStatements i ResultSet, aby zapobiec wyciekowi pamięci, jak wyjaśniono tutaj Jednak ostatnio zacząłem pracować z nowym zespołem, który używa połączenia org.sql2o.connection, co jest miłym i użytecznym opakowaniem jdbc, ale zauważyłem, że zwykle nie zamykają swoich obiektów org.sql2o.Query. try (org.sql2o.Connection c = Sql2oObject.open ()) { //...some logika Listlist = c.createQuery (sql) .executeAndFetch (SomeClass.class); //...s trochę więcej logiki } Jak widać, obiekt zapytania nigdy nie jest jawnie zamykany. Myślę, że lepiej byłoby zamknąć zapytanie, jak również takie: try (org.sql2o.Connection c = sql2oObject.open (); org.sql2o.Query q = c.createQuery (sql); ) { //...some logika List list = q.executeAndFetch (SomeClass.class); //...s trochę więcej logiki } Ale nie jestem pewien, czy to naprawdę konieczne. Jeśli tak, to znalazłem duży prawdopodobny wyciek i będę uważany za geniusza i zapamiętany na wieczność. Jeśli nie, to jestem po prostu facetem proszącym o niepotrzebną zmianę. Więc co powiesz o potężny SO? czy konieczne jest ich zamknięcie?
2021-02-01 08:19:23
Naprawdę nie musisz wywoływać close, ponieważ używasz sql2o.connection, ponieważ używasz go z podejściem try-with-resource w Javie. To, co robi w java try-with-resource, polega na tym, że kompilator dodaje instrukcję final do twojego kodu, w której zakłada się jawne zamknięcie zasobu natywnego. W przeciwnym razie musisz zamknąć połączenie sql2o wewnątrz bloku last. Ponieważ jest to opakowanie jdbc wokół standardowego połączenia jdbc Jeśli chcesz zamknąć również inne podkreślone zasoby za pomocą try-with-resource, zadeklaruj je wszystkie w nawiasach. Aby zamknęło podkreślone zasoby razem z głównym zasobem. Jednak w twoim przypadku zapytanie nie musi być jawnie zamykane, ponieważ zawsze jest częścią połączenia sql2o.connection | Twoja odpowiedź StackExchange.ifUsing ("editor", function () { StackExchange.using ("externalEditor", function () { StackExchange.using ("snippets", function () { StackExchange.snippets.init (); }); }); }, "fragmenty kodu"); StackExchange.ready (function () { var channelOptions = { tagi: „” .split („”), id: "1" }; initTagRenderer ("". split (""), "" .split (""), channelOptions); StackExchange.using ("externalEditor", function () { // Muszę uruchomić edytor po fragmentach, jeśli fragmenty są włączone if (StackExchange.settings.snippets.snippetsEnabled) { StackExchange.using ("snippets", function () { createEditor (); }); } else { createEditor (); } }); function createEditor () { StackExchange.prepareEditor ({ useStacksEditor: false, heartbeatType: 'answer', autoActivateHeartbeat: false, convertImagesToLinks: true, noModals: true, showLowRepImageUploadWarning: true, reputationToPostImages: 10, bindNavPrevention: prawda, przyrostek: "", imageUploader: { brandingHtml: „Obsługiwane przez \ u003ca href = \" https: //imgur.com/ \ "\ u003e \ u003csvg class = \" ikona-svg \ "width = \" 50 \ "height = \" 18 \ "viewBox = \ "0 0 50 18 \" fill = \ "none \" xmlns = \ "http: //www.w3.org/2000/svg \" \ u003e \ u003cpath d = \ "M46.1709 9.17788C46.1709 8.26454 46.2665 7,94324 47,1084 7,58816C47.4091 7,46349 47,7169 7,36433 48,0099 7,26993C48.9099 6,97997 49,672 6,73443 49,672 5,93063C49.672 5,22043 48,9832 4,61182 48,1414 4,61182C47.4335 4,61182 46,7256 4,9256,43,43354,61182 46,7256 4,9168,60 468,56 43.1481 6.59048V11.9512C43.1481 13.2535 43.6264 13.8962 44.6595 13.8962C45.6924 13.8962 46,1709 13.2535 46,1709 11.9512V9.17788Z \ "/ \ u003e \ u003cpath d = \" M32.492 10.1419C4184184.492 1448.1709 1448.1709 41,5985 12,6954 41,5985 10,1419V6,59049C41,5985 5,28821 41,1394 4,66232 40,1061 4,66232C39.0732 4,66232 38,5948 5,28821 38,5948 6,59049V9.60062C38,5948 10,8521 38,2696 11,5455 37,0451 11,5455C35 521 35,4954 9,60062V6,59049C35,4954 5,28821 35,0173 4,66232 34,0034 4,66232C32,9703 4,66232 32,492 5,28821 32,492 6,59049V10,1419Z \ "/ \ u003e \ u003cpath fill-rule = \" evenodd \ "clip-rule = \" evenodd \ "clip-rule = \" = \ "M25.6622 17.6335C27.8049 17.6335 29.3739 16.9402 30.2537 15.6379C30.8468 14.7755 30.9615 13.5579 30.9615 11.9512V6.59049C30.9615 5.28821 30.4833 4.66231 29.4502 4.66231C28.9913 4.663.4.865 C287.65. .1369 4,56087 21,0134 6,57349 21,0134 9,27932C21.0134 11,9852 23,003 13,913 25,3754 13,913C26,5612 13,913 27,4607 13,4902 28,1109 12,6616C28.1109 12,7229 28.1161 12,7799 28,121 12,8341C28.1256 12,8341308,1253 28,1302,83 28,1302,130,130,130,125,8341301 15.2321 24.1352 14.9821 23.5661 14.7787C23.176 14.6393 22.8472 14.5218 22.5437 14.5218C21.7977 14.5218 21.2429 15.0123 21.2429 15.6887C21.2429 16.7375 22.9072 17.6335 25.6622 17.6335ZM97 24.6335ZM97 24.6335ZM24. 27.2119 7.09766 28.0918 7.94324 28.0918 9.27932C28.0918 10.6321 27.2311 11.5116 26.1024 11.5116C24.9737 11.5116 24.1317 10.6491 24.1317 9.27932Z \ "/ \ u003e \ u003cpath d = \" M16.8045 11,95612C1962 13,862 13,862 13,862. 19,8079 13,2535 19,8079 11,9512V8.12928C19.8079 5,82936 18,4879 4,62866 16,4027 4,62866C15,1594 4,62866 14,279 4,98375 13,3609 5,88013C12,653 5,05154 11,6581 4,62866 10,3573 4,62866C9.34336 4,62866 8,5766C9,34336 4,62866 8,5766C9,34336 4,62866 8,57664,83,63 4,86603 4,86603 4,66,66 4,83,63 4,86164,86164,66 6,66 4,86163 4,96 6,63 4,96 6,63 4,96 4,66 4,86 6,67 5,00066 5,28821 5,00066 5,47873 13,8962 6.59049V11.9512C5.00066 13,2535 6,51203 13.8962C7.54479 13,8962 8,0232 13,2535 8,0232 11.9512V8.90741C8.0232 7,58817 8,44431 6,91179 9,53458 7,58817 6.91179C10.5104 6,91179 10,893 10,893 8.94108V11.9512C10.893 13,2535 11,3711 13,8962 12,4044 13,8962 C13.4375 13,8962 13,9157 13,2535 13,9157 11,9512V8.90741C13.9157 7,58817 14,3365 6,91179 15,4269 6,91179C16,4027 6,91179 16,8045 7,58817 16,8045 8,94108V11,9512Z \ "/ \ u003e \ u003cpath d = \ "M3.31675 6,59049C3.31675 5,28821 2,83866 4,66232 1,82471 4,66232C0,791758 4,66232 0,313354 5,28821 0,313354 6,59049V11.9512C0.313354 13,2535 0,791758 13,8962 1,82471 13,8962C2,8573511,9512V6,59049Z \ "/ \ u003e \ u003cpath d = \" M1,87209 0,400291C0,843612 0,400291 0 1,1159 0 1,98861C0 2,87869 0,822846 3,57676 1,87209 3,57676C2,90056 3,57676 3,7234 2,87869 3,7234 1,98291 2,48291 1,98291 2,48291 1,98291 1,98291 2,6234 \ "fill = \" # 1BB76E \ "/ \ u003e \ u003c / svg \ u003e \ u003c / a \ u003e", contentPolicyHtml: "Wkład użytkowników licencjonowany na mocy \ u003ca href = \" https: //stackoverflow.com/help/licensing \ "\ u003ecc by-sa \ u003c / a \ u003e \ u003ca href = \" https://stackoverflow.com / legal / content-policy \ "\ u003e (polityka treści) \ u003c / a \ u003e", allowUrls: true }, onDemand: prawda, discardSelector: ".discard-answer" , natychmiastShowMarkdownHelp: true, enableTables: true, enableSnippets: true }); } }); Dziękujemy za przesłanie odpowiedzi na temat Stack Overflow! Pamiętaj, aby odpowiedzieć na pytanie. Podaj szczegóły i udostępnij swoje badania! Ale unikaj… Proszenie o pomoc, wyjaśnienia lub odpowiadanie na inne odpowiedzi. Wygłaszanie oświadczeń na podstawie opinii; poprzyj je referencjami lub osobistym doświadczeniem. Aby dowiedzieć się więcej, zapoznaj się z naszymi wskazówkami dotyczącymi pisania świetnych odpowiedzi. Wersja robocza zapisana Wersja robocza odrzucona Zarejestruj się lub zaloguj StackExchange.ready (function () { StackExchange.helpers.onClickDraftSave ('# login-link'); }); Zarejestruj się za pomocą Google Zarejestruj się za pomocą Facebooka Zarejestruj się przy użyciu adresu e-mail i hasła Zatwierdź Opublikuj jako gość Nazwa E-mail Wymagane, ale nigdy nie pokazywane StackExchange.ready ( function () { StackExchange.openid.initPostLogin ('. New-post-login', 'https% 3a% 2f% 2fstackoverflow.com% 2fquestions% 2f64796956% 2fdo-i-need-to-close-sql2o-queries% 23new-answer', ' question_page '); } ); Opublikuj jako gość Nazwa E-mail Wymagane, ale nigdy nie pokazywane Opublikuj swoją odpowiedź Odrzucać Klikając „Opublikuj odpowiedź”, akceptujesz nasze warunki korzystania z usługi, politykę prywatności i politykę dotyczącą plików cookie Nie szukasz odpowiedzi? Przeglądaj inne pytania otagowane java jdbc garbage-collection sql2o lub zadaj własne pytanie.